第17回 svgicon のパッチレビュー

PR:

問題点 1

svg がインラインに展開されるのでキャッシュが効かない

問題点 2

icon helper の labelled_icon というメソッドと、html のクラス (class="icon icon-edit") という2箇所で icon に関する指定が必要。冗長な感じがする

解決方法

svg の use 要素で外部のsvg を読み込み可能 href を使用すれば良い？

https://tsuchippo.com/htmlcss-svgcolor では、xlink:href で読みこんでいるけど、現在は非推奨らしい。

<svg>

</svg>

MariusさんのパッチはPOCのような形で、Issueビューの上の部分だけのもの

問題点1については、SVGファイル内を<symbol>要素で囲むことで、外部リンクとして読み込める形で対応

IE11はSVGの外部読み込みには対応してなさそうだが、本家側でのIE11ドロップに期待。

取り急ぎ、参考実装し、パッチを該当Issueに送付予定。

labelled_icon メソッドは冗長なので、 link_to か、 content_tag に修正を加え、 icon: :edit, svg: true とかオプションをつけるだけで、svg アイコンが表示されるようになるとかっこいいのではないか？

現状、svg アイコンがやや上に表示される気がする。

現状=>SVGアイコン移行パスとして、現状アイコンとSVGアイコンが設定で切り替えられると良いかも。